Skip to content

Conversation

@dbermond
Copy link
Contributor

@dbermond dbermond commented May 10, 2025

For using fixed width integer types, the <cstdint> C++ header needs to be explicitly included with GCC 15 due to changes in libstdc++.

For details, see the documentation about porting to GCC 15.

@JablonskiMateusz
Copy link
Contributor

Hi @dbermond

Thanks for contribution!
Please correct commit msg format according to https://github.com/intel/compute-runtime/blob/master/CONTRIBUTING.md#commit-message-format

@dbermond dbermond changed the title Add fixes for building with GCC 15 build: add missing headers for GCC 15 May 17, 2025
@dbermond
Copy link
Contributor Author

dbermond commented May 17, 2025

Thanks for contribution! Please correct commit msg format according to https://github.com/intel/compute-runtime/blob/master/CONTRIBUTING.md#commit-message-format

Commit message updated. Hope this is fine now. Please let me know if something else needs to be changed.

@JablonskiMateusz
Copy link
Contributor

Thanks for contribution! Please correct commit msg format according to https://github.com/intel/compute-runtime/blob/master/CONTRIBUTING.md#commit-message-format

Commit message updated. Hope this is fine now. Please let me know if something else needs to be changed.

Please update year in copyright headers in modified files

For using fixed width integer types[1], the `<cstdint>`[2]
C++ header needs to be explicitly included with GCC 15 due
to changes[3] in libstdc++.

For details, see the documentation[4] about porting to GCC 15.

[1] https://en.cppreference.com/w/cpp/types/integer
[2] https://en.cppreference.com/w/cpp/header/cstdint
[3] https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=3a817a4a5a6d94da9127af3be9f84a74e3076ee2
[4] https://gcc.gnu.org/gcc-15/porting_to.html#cxx

Signed-off-by: Daniel Bermond <dbermond@archlinux.org>
@dbermond
Copy link
Contributor Author

Please update year in copyright headers in modified files

Done.

@JablonskiMateusz
Copy link
Contributor

merged in e0362a7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged change was merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants